-
-
Notifications
You must be signed in to change notification settings - Fork 120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a workaround to support VS Code's Remote WSL extension (fix #238) #324
Conversation
I would prefer to see something like https://github.com/K900/vscode-remote-workaround, which simply uses nixpkgs node.js |
This PR is technically not a workaround. It just provides what vscode-server assumes (global C runtime and global coreutils). |
Then neither is my approach, because it provides a functioning Node runtime. |
But it replaces vscode’s node on fly, right?
…On Tue, Nov 7, 2023 at 7:29 PM K900 ***@***.***> wrote:
Then neither is my approach, because it provides a functioning Node
runtime.
—
Reply to this email directly, view it on GitHub
<#324 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAES3OVQE52LJSADH3D26ELYDL4CRAVCNFSM6AAAAAA634FOO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBQHE2TIMZXGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Correct. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this affects every program and can cause unexpected regressions I am very reluctant to accept it, especially regarding that we would still need an older WSL release which is probably a separate problem but is helping the situation not.
description = "Workaround for VSCode's Remote WSL extension"; | ||
}; | ||
nix-ld-rs = lib.mkOption { | ||
type = lib.types.raw; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be type package.
Since you are reluctant to accept this PR. I just put it in a separate repository and close this PR. Feel free to use this module from https://github.com/Atry/nixos-wsl-vscode |
This PR adds
modules/vscode-remote-wsl-extension-workaround.nix
, whenwsl.vscodeRemoteWslExtensionWorkaround.enable
istrue
, VS Code's Remote WSL extension can connect to NixOS, fixing #238See https://github.com/Atry/nixos-wsl-vscode/blob/main/flake.nix for usage